✨ Feat: 프레임 컴포넌트 구현 #1 #8
Conversation
feat: 프레임 컴포넌트 구현 codeit-FE18-part3#1
|
@BaeZzi813 is attempting to deploy a commit to the projects Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
sohyun0
left a comment
sohyun0
left a comment
There was a problem hiding this comment.
고생하셨습니다!
코멘트 확인부탁드립니다 :)
폴더구조를 src/components/layout/frame/frame.tsx 으로 가져가면 좋을것 같습니다!
-> ui 보다는 레이아웃의 성격이 조금더 강해서 추천드립니다.
-> frame.tsx 파일 한개만 있는데 너무 과한 폴더구조가 발생되어서 이것때문에 카테고리별 분류를 조금 우려했던건데 일단 해당 컨벤션으로 작성 부탁드립니다! (추후 수정을하는 방향으로 가봐요)
| <button | ||
| onClick={() => router.push(`/${address}`)} |
There was a problem hiding this comment.
어느 페이지로 이동하는거면 Next 컴포넌트의 Link 를 사용하는것도 좋은 방법일것 같습니다.
또한 네이밍도 address 는 일반 주소를 의미하는 뜻이 강해 navigate / route / href 와 같은 패스형태의 네이밍도 추천드립니다 :)
| <h2 className='text-sm font-normal text-black tablet:text-base'>{content}</h2> | ||
| <button | ||
| onClick={() => router.push(`/${address}`)} | ||
| className='rounded-md bg-[#ea3c12] px-5 py-[10px] text-sm font-bold text-white tablet:px-[136px] tablet:py-[14px] tablet:text-base' |
There was a problem hiding this comment.
버튼 스타일은 추후 신천님께서 버튼 컴포넌트 제작하면 리팩토링 하셔도 좋을것 같습니다.
또한 #EA3C12 컬러는 팔레트에 없던 컬러여서 미쳐 못봤었는데 이것도 red-500으로 컨피그 넣어놓겠습니다 :)
|
리뷰 감사합니다~ 말씀하신대로 폴더 옮겨놓고 Link 수정 후 다시 PR 올리면 될까요~? |
|
2 participants
feat: 프레임 컴포넌트 구현 #1
📝 작업 개요 (필수)
✨ 작업 내용 (필수)
📸 스크린샷
🧐 해결해야 하는 문제
🤔 리뷰어 확인 필요 사항
🔗 관련 이슈
🛠️ 후속 작업
✅ 체크리스트 (필수)